Skip to content

fix: preserve todos after stream completes#3314

Open
18062706139fcz wants to merge 1 commit into
bytedance:mainfrom
18062706139fcz:fix/todo-persist-3271
Open

fix: preserve todos after stream completes#3314
18062706139fcz wants to merge 1 commit into
bytedance:mainfrom
18062706139fcz:fix/todo-persist-3271

Conversation

@18062706139fcz
Copy link
Copy Markdown
Contributor

Summary

  • preserve todos when a later thread state update omits the todos field
  • keep explicit empty todo lists as a valid clear operation
  • add regression tests for omitted-vs-empty todo state updates

Verification

  • pnpm exec vitest run tests/unit/core/threads/message-merge.test.ts
  • pnpm exec eslint src/core/threads/hooks.ts tests/unit/core/threads/message-merge.test.ts
  • pnpm typecheck

Closes #3271

@CLAassistant
Copy link
Copy Markdown

CLAassistant commented May 29, 2026

CLA assistant check
All committers have signed the CLA.

@18062706139fcz 18062706139fcz force-pushed the fix/todo-persist-3271 branch from bf4e48a to 3926b61 Compare May 29, 2026 06:56
@18062706139fcz 18062706139fcz force-pushed the fix/todo-persist-3271 branch from 3926b61 to 22d66d6 Compare June 4, 2026 03:18
@github-actions github-actions Bot added needs-validation Touches front/back contract surface; needs real-path validation risk:medium Medium risk: regular code changes size/S PR changes 20-100 lines area:frontend Next.js frontend under frontend/ and removed size/S PR changes 20-100 lines risk:medium Medium risk: regular code changes needs-validation Touches front/back contract surface; needs real-path validation labels Jun 4, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area:frontend Next.js frontend under frontend/

Projects

None yet

Development

Successfully merging this pull request may close these issues.

To-dos 待办事项在流式传输期间可见,但在运行完成后会消失。在下一次对话流式传输期间又可见。

2 participants